This repository was archived by the owner on Jan 26, 2021. It is now read-only.
Addition of updated automated tests to vms #315
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We need to document the failed test cases as well. How many are failing? |
|
There are 8 Failures. These are due to issue #327 :
I forgot to mention, there would be one error - in test_volunteer_cannot_access_admin_urls |
|
I tested it. Good. :) For me it ran 150 tests in 1829.783s with FAILED (failures=8, errors=1). |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is to add updated automated tests to the VMS repository.
Earlier, the automated tests were in a separate repository and vms just had the unit tests.
The first three commits in this PR add the previous tests to the vms repository, move unit tests to the 'tests' folder so as to avoid conflicts and add small modifications to the installation guide so that a user can run unit tests or a test file separately if they want. The version of Selenium has also been upgraded to the latest one.
The packages mentioned in requirements.txt file need to be installed as a prerequisite. Also, firefox and selenium version compatibility has to be checked.
Right now,
pip install selenium==2.53.1is run by default. For another selenium version, run the same command replacing 2.53.1 with the version compatible with the users firefox browser. My Firefox version is 44.0.2 which is compatible with 2.53.1.For finding out FIrefox version - here
The tests added previously in the last year have been run against the vms codebase in August 2015. Since vms has changed in the last one year, 76 out of total 101 cases failed or showed up errors after running them against the latest code. The last 5 commits in this PR are to update these tests app wise to resolve the errors.
After merging this PR, there should not be any errors in unit or functional tests, although a few failed cases would still be reported. Tests across all apps have been updated and errors resolved as per latest changes in vms codebase. All the tests for each app, whether it is unit or functional would be placed in the same tests folder in the vms repository.
To run all tests, run the following command :
python manage.py test